Skip to content

RSDK-7359: remove deprecated robot_part_id from cloudMetadataResponse#830

Open
allisonschiang wants to merge 2 commits intoviamrobotics:mainfrom
allisonschiang:RSDK-7359
Open

RSDK-7359: remove deprecated robot_part_id from cloudMetadataResponse#830
allisonschiang wants to merge 2 commits intoviamrobotics:mainfrom
allisonschiang:RSDK-7359

Conversation

@allisonschiang
Copy link
Member

No description provided.

@github-actions github-actions bot added the safe to test committer is a member of this org label Mar 19, 2026
@allisonschiang allisonschiang added allow breaking Allow breaking changes by bypassing the buf-breaking check protos-compiled labels Mar 19, 2026
@allisonschiang allisonschiang requested a review from cheukt March 19, 2026 20:58
string location_id = 3;
string machine_id = 4;
string machine_part_id = 5;

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do I need to make an announcement about this in breaking changes? since it was deprecated 2 years ago I'd hope it doesn't affect anything but I haven't gone through the deprecation process before

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep you should - in case anyone is still relying on this field

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will approve after announcement made

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do I need to give people time(one week?) before merging this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow breaking Allow breaking changes by bypassing the buf-breaking check safe to test committer is a member of this org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants